Skip to content

fix disign issue#512

Merged
dvCodeWorld merged 1 commit intodevelopmentfrom
P0080-clean
Apr 1, 2026
Merged

fix disign issue#512
dvCodeWorld merged 1 commit intodevelopmentfrom
P0080-clean

Conversation

@Saurabh186
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown

@windsurf-bot windsurf-bot bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 To request another review, post a new comment with "/windsurf-review".

[matTooltip]="element.reference_id"
matTooltipPosition="above"
class="cursor-pointer font-mono text-xs px-2 py-0.5 rounded-4 min-w-[150px]"
class="cursor-default"
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change removes several styling classes (font-mono text-xs px-2 py-0.5 rounded-4 min-w-[150px]) that affect the appearance of the reference ID display. Was this intentional? If the goal was just to change the cursor style, you might want to keep the other styling classes to maintain the visual design.

<!-- ── Snippet Tab ── -->
@if (selectedSectionIndex === 2) {
<div class="flex flex-col gap-4 max-w-2xl">
<div class="flex flex-col gap-4">
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing the max-width constraint from the snippet container could make code snippets stretch too wide on larger screens, reducing readability. Consider keeping a reasonable maximum width.

<div class="card-with-header--header justify-between">
<div class="flex items-center gap-2">
<mat-icon class="!text-base !w-4 !h-4 secondary-text-color opacity-70">html</mat-icon>
<mat-icon>html</mat-icon>
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The HTML icon is missing the same sizing classes that are applied to the code icon above it. This will cause inconsistent icon sizing in the UI.

Suggested change
<mat-icon>html</mat-icon>
<mat-icon class="!text-base !w-4 mt-1">html</mat-icon>

@dvCodeWorld dvCodeWorld merged commit 76a8b35 into development Apr 1, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants